home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 317 < prev    next >
Encoding:
Text File  |  1996-08-05  |  2.7 KB  |  108 lines
  1. Newsgroups: alt.msdos.programmer,comp.lang.c
  2. Path: news.kei.com!wang!news
  3. From: emild@cs.technion.ac.il (Kohn Emil Dan)
  4. Subject: Re: Pascal program works but not C program! 
  5. Organization: Technion, Israel Institute of Technology
  6. Date: Thu, 4 Jan 1996 10:18:15 GMT
  7. Message-ID: <Pine.SV4.3.91-heb-2.04.960104114930.16292A-100000@cs.technion.ac.il>
  8. References: <4cdpr2$psi@lugb.latrobe.edu.au> 
  9. Sender: news@wang.com
  10.  
  11.  
  12.  
  13. On 3 Jan 1996, Gregary John Boyles wrote:
  14.  
  15. > EMail : boyles@lux.latrobe.edu.au
  16. >       
  17. > Why does the Pascal version of this program behave exactly as expected while the
  18. > C++ version does not?
  19. > 
  20. > 
  21. > 
  22. > 
  23. > ******************************* PASCAL VERSION *******************************
  24. > 
  25. /*Pascal  code removed*/ 
  26. > 
  27. > ******************************** C++ VERSION *********************************
  28. > 
  29. > #include <stdio.h>
  30. > #include <conio.h>
  31. > #include <string.h>
  32. > #include <alloc.h>
  33. > #include <process.h>
  34. > 
  35. > const maxstring=127;
  36. > const lineend='\n';
  37. > const escape=27;
  38. > const null='\0';
  39. > const space=' ';
  40. > const up='H';
  41. > const down='P';
  42. > const pageup='I';
  43. > const pagedown='Q';
  44. > const left='K';
  45. > const right='M';
  46. > const home='G';
  47. > const end_='O';
  48. > 
  49. > 
  50. > 
  51. > void writescreenfull(char width,char height,nodetypeptr posptr,char stringpos)
  52. > {
  53. >    char y;
  54. > 
  55. >    window(1,1,width,height);
  56. >    clrscr();
  57. >    window(1,1,width,height+1);
  58. >    y=1;
  59. >    while ((y<=height) && (!empty(posptr->next)))
  60. >    {
  61. >         gotoxy(1,y);
  62. >         strcpy(line_,posptr->line);
  63. >         line__=&(line_[stringpos]);
  64. >         strncpy(line,line__,width-1);
  65. >         strcat(line,null);
  66.                        ^---------------This is a problem!!!!!!!!!
  67.                        
  68. >         cputs(line);
  69. >         posptr=posptr->next;
  70. >         y++;
  71. >    }
  72. > }
  73. > 
  74. > 
  75. > void main(int argc,char *argv[])
  76.    ^-----------------------------This one too, but not too serious
  77. > {
  78. > }
  79. > 
  80. > 
  81.  
  82.  
  83. I hadn't read  your code thoroughly, but I think your problem is with 
  84. strcat (I mean surely there is a problem with the usage of strcat, but I 
  85. don't know if it's the only one). In C (and in C++ too), a string of length 
  86. 1 is _NOT_ the same thing as a character. So null (as defined by you) is _NOT_ a string.
  87. An empty string is defined as "". If you want to append a '\0' string 
  88. terminator to line_  (I think this was your intention), use the statement:
  89.  
  90. line_[width-1]='\0'; (or using your constant line_[width-1]=null;) 
  91.  
  92. Don't bother, if the length of line__ is less than width-1; strncpy padds 
  93. the destination string with '\0'-s.
  94.  
  95.  
  96.  
  97. Another problem, not as serious as the previous one, although this is not
  98. the reason the C++ version might not work; main() should be defined
  99. returning int, not void. 
  100.  
  101.  
  102. Hope this helps.
  103.  
  104.  
  105.                     Best regards, 
  106.  
  107.                                 Emil
  108.